Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rds_instance: Set no_log=False on force_update_password #244

Merged
merged 3 commits into from
Nov 10, 2020

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Sep 24, 2020

SUMMARY

Because force_update_password doesn't contain sensitive information (but matches the pattern "password") we need to explicitly set no_log=False to avoid a warning for users.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

rds_instance

ADDITIONAL INFORMATION

fixes: #241

@tremble tremble requested a review from jillr September 30, 2020 09:04
@tremble
Copy link
Contributor Author

tremble commented Sep 30, 2020

@jillr mind giving this one a quick review - be good to remove the warning before the next release.

@felixfontein felixfontein merged commit f47987b into ansible-collections:main Nov 10, 2020
@felixfontein
Copy link
Contributor

@tremble thanks for fixing this!
@markuman thanks for reviewing!

@tremble tremble deleted the issues/241 branch February 11, 2021 09:14
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
rds_instance: Set no_log=False on force_update_password
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
rds_instance: Set no_log=False on force_update_password
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rds_instance should explicitly set no_log=False for force_update_password
3 participants